Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove outdated comment #10853

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Jul 24, 2023

Since support for persistence 2 has been dropped, this method may no longer acces an aliased class name.
Besides, providing an FQCN with a leading backslash should work since removing it is the first thing that happens inside AbstractClassMetadataFactory::getMetadataFor(): https://github.com/doctrine/persistence/blob/413eb71a22c31c309b5a1cea9701d723cc0d4ae2/src/Persistence/Mapping/AbstractClassMetadataFactory.php#L164-L177

Since support for persistence 2 has been dropped, this method may no
longer acces an aliased class name.
Besides, providing an FQCN with a leading backslash should work since
removing it is the first thing that happens inside
AbstractClassMetadataFactory::getMetadataFor().
@derrabus derrabus added this to the 3.0.0 milestone Jul 24, 2023
@derrabus derrabus merged commit 2dc19e6 into doctrine:3.0.x Jul 24, 2023
@greg0ire greg0ire deleted the remove-outdated-comment branch July 25, 2023 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants